Skip to content

Conversation

@elsazac
Copy link
Member

@elsazac elsazac commented Oct 23, 2025

This creates a filter text to the Templates preference pages for providing filtering option. This change adds a filter text field to the Templates page UI. This introduces filtering across multiple templates such as Java, Ant, and Run/Debug template preference pages.

templates.mov

@elsazac elsazac force-pushed the templatesFiltering branch 2 times, most recently from 86cad82 to 7494608 Compare October 23, 2025 11:59
@github-actions
Copy link
Contributor

github-actions bot commented Oct 23, 2025

Test Results

 3 018 files  ±0   3 018 suites  ±0   2h 21m 12s ⏱️ - 1m 52s
 8 234 tests ±0   7 985 ✅ ±0  249 💤 ±0  0 ❌ ±0 
23 622 runs  ±0  22 828 ✅ ±0  794 💤 ±0  0 ❌ ±0 

Results for commit e4080ed. ± Comparison against base commit 3a441d9.

♻️ This comment has been updated with latest results.

@BeckerWdf
Copy link
Member

Is this then also visible in the "Templates" view?

@elsazac
Copy link
Member Author

elsazac commented Oct 24, 2025

Is this then also visible in the "Templates" view?

This is affecting not the view, but only the templates preference page (Preferences → Java → Editor → Templates)
The Templates View is a completely separate component.

Screenshot 2025-10-24 at 11 30 37 AM Screenshot 2025-10-24 at 11 32 10 AM

@elsazac elsazac force-pushed the templatesFiltering branch from 7494608 to 6c2df08 Compare October 24, 2025 06:05
@BeckerWdf
Copy link
Member

The Templates View is a completely separate component.

Ok. Just wanted to be sure.

@elsazac elsazac force-pushed the templatesFiltering branch 3 times, most recently from b50c3d0 to df23463 Compare October 30, 2025 13:43
@elsazac elsazac force-pushed the templatesFiltering branch from df23463 to e4080ed Compare November 2, 2025 13:43
@elsazac
Copy link
Member Author

elsazac commented Nov 3, 2025

@BeckerWdf Do you have any outstanding concerns on this PR, else I would like to merge this.

@elsazac elsazac merged commit 3ee82a4 into eclipse-platform:master Nov 3, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants